-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Solid to react dev #37
base: main
Are you sure you want to change the base?
Conversation
- Bump vite to 4.5.2 to solve security issue reported by dependabot in https://github.com/hyperledger/cacti/pull/2999 - Update other vite and build plugins to latest versions. - Update solidjs and solidjs router to fix runtime issues. Signed-off-by: Michal Bajer <[email protected]>
@@ -0,0 +1,18 @@ | |||
module.exports = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Usun config eslinta, uzywaj tego z roota
@@ -0,0 +1,24 @@ | |||
# Logs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To samo z gitignore, jesli cos z tego nie jest ignorowane to dodaj w glowny .gitignore
@@ -0,0 +1,30 @@ | |||
# React + TypeScript + Vite |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
README wiadomo zaktualizowac z tym co bylo
<meta charset="UTF-8" /> | ||
<link rel="icon" type="image/svg+xml" href="/vite.svg" /> | ||
<meta name="viewport" content="width=device-width, initial-scale=1.0" /> | ||
<title>Vite + React + TS</title> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tytul / ikonka do zmiany
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i wywal package.lock, wszystko yarnem powinno sie instalowac tak jak to jest w obecnej paczce
}, | ||
"type": "module", | ||
"scripts": { | ||
"dev": "vite", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
skrypty musza byc takie jak w poprzendiej paczce, w szcegolnosci build:*:frontend
function CustomTable(props) { | ||
|
||
|
||
const [viewport, setViewport] = useState(""); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tutaj cos duzo tych spacji po funkcji, odpal linta przed PR zeby porpawil wszystkie bledy takie
@@ -0,0 +1,68 @@ | |||
/* :root { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
usun jak nie potrzebne
]); | ||
|
||
ReactDOM.createRoot(document.getElementById("root")!).render( | ||
<React.StrictMode> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
strict mode mozesz zakomentowac przed PR
import { useNavigate } from "react-router-dom"; | ||
import { useEffect, useState } from "react"; | ||
import CardWrapper from "../../../components/CardWrapper/CardWrapper"; | ||
// import { Transaction } from "../../../schema/supabase-types"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
usun zakomentowane importy
export default FabricBlock; | ||
|
||
{ | ||
/* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
to do usuniecia chyba
Pull Request Requirements
upstream/main
branch and squashed into single commit to help maintainers review it more efficient and to avoid spaghetti git commit graphs that obfuscate which commit did exactly what change, when and, why.-s
flag when usinggit commit
command. You may refer to this link for more information.Character Limit
A Must Read for Beginners
For rebasing and squashing, here's a must read guide for beginners.